docs(studies): carousel and pager candidate study - #25
Draft
k08200 wants to merge 4 commits into
Draft
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
k08200
force-pushed
the
docs/study-carousel-pager
branch
from
July 21, 2026 14:51
a4e519e to
fd5cc6f
Compare
This was referenced Jul 22, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Adds
docs/studies/carousel-and-pager.md— a candidate study for the "Carousel and existing pager" entry in the widget candidate landscape (#19) — plus its index line indocs/studies/README.md.Docs only; no code, no syntax reservation, no obligation created for the future
uisub-specification (per the studies charter).Method
mainand the feat!: replace Uhura with the 0.4 machine-first language #24 branch, updated to its current head7f69fc0(spike catalog contract,ui_catalogvocabulary, the pager primitive adapter and its tests).Findings the study rests on
7f69fc0, the pager primitive now dispatchespage-change— with aUnitpayload, fired on rounded-index crossings during scrolling. The machine learns that the page changed but not which page; the current page itself stays renderer-internal, and no example binds the event on either branch. The position half of the contract — whether, when, and in what vocabulary a program may own the current page — is the open question the study addresses.Candidate answers (summary)
PageChangedat rest) once anything observes it — the textfield rule applied to paging.pager+button(semantic controls + rotation policy), not a separate Element — conditional on whichever composition tier theuisub-specification defines.Open questions the study cannot decide are listed at the end for the
uisub-specification.Closes nothing; intended as input for the future
uisub-specification. Feedback welcome — happy to rework or drop per your direction.